-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
BUG: pandas.cut and negative values #14652 #14663
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
tests pls |
pls add an entry in whatsnew for 0.19.2 |
Current coverage is 85.28% (diff: 100%)@@ master #14663 diff @@
==========================================
Files 140 140
Lines 50693 50693
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 43234 43234
Misses 7459 7459
Partials 0 0
|
Please let me know if that's ok because I wasn't sure where to add the test case. |
@@ -25,3 +25,4 @@ Bug Fixes | |||
|
|||
- compat with ``dateutil==2.6.0`` for testing (:issue:`14621`) | |||
- allow ``nanoseconds`` in ``Timestamp.replace`` kwargs (:issue:`14621`) | |||
- fix ``pandas.cut`` with one negative value and one bin (:issue:`14652`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug in pd.cut
|
||
expected = pd.Series([0, 0]) | ||
|
||
s = pd.Series([9., 9.]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pandas\tools\tests\test_tile.py
lots of tests there, just add this one on., no need to create a new class. add a comment with the issue number
Thanks @jreback, now it should be fine. |
thanks! |
…s-dev#14652 closes pandas-dev#14652 Author: Luca Scarabello <luca.scarabello@sed.ethz.ch> Author: Luca <luca.scarabello@gmail.com> Closes pandas-dev#14663 from luca-s/issue_14652 and squashes the following commits: 8db26db [Luca Scarabello] Moved new test location to pandas\tools\tests\test_tile.py 90dd07d [Luca Scarabello] Updated whatsnew d6b3da8 [Luca Scarabello] fixed flake8 compliance fdc55b9 [Luca Scarabello] Added test case for pandas-dev#14652 d5790e2 [Luca Scarabello] updated whatsnew v0.19.2 2db0c7a [Luca] BUG: pandas.cut and negative values pandas-dev#14652 (cherry picked from commit 2fc0c68)
git diff upstream/master | flake8 --diff